ci: Gradle の更新設定をディレクトリ横断のグループへまとめる - #41
Merged
Merged
Conversation
integration-test の settings はルートビルドを composite 参照するため、 /integration-test の更新ジョブが見るマニフェストにはルート側（gradle wrapper 等）が 含まれる。ディレクトリごとに更新設定を分けていたため、同一の更新が二重に PR 化され、 かつ広い範囲を更新する側の PR 表題が /integration-test という狭い範囲を指していた。 両ディレクトリを1つの更新設定へまとめ、group-by: dependency-name によって 依存名ごとの単一 PR へ束ねる。 Co-Authored-By: Claude Opus 5 <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
背景
Gradle の更新設定を
/と/integration-testの2つに分けていたが、integration-test/settings.gradle.ktsがincludeBuild("..")でルートビルドを composite 参照するため、/integration-testの更新ジョブが見るマニフェストにはルート側（gradle wrapper 等）が含まれる。結果として #38 と #39 が同一の gradle-wrapper 9.6.1 → 9.7.0 に対する重複 PR となり、
さらに広い範囲（ルート＋integration-test の両 wrapper）を更新する #38 の表題が
in /integration-testという狭い範囲を指していた。/integration-test... in /integration-test/...（範囲表記なし）変更
両ディレクトリを1つの更新設定（
directories）へまとめ、group-by: dependency-nameで依存名ごとの単一 PR へ束ねる。依存ごとに PR が分かれる粒度は維持したまま、
ディレクトリ違いによる重複が解消し、PR の対象範囲と表題も一致する。
org.apache.maven:*の ignore は統合後の設定が引き継ぎ、integration-test 側のジョブにも適用される（maven 版はルートの catalog を共有するため意図どおり）。
確認
group-byは version updates 向けの GA 機能で、対象ディレクトリが同一エコシステムであることが条件（本設定は両方とも gradle）🤖 Generated with Claude Code